Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding a new service documentation #73

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Adding a new service documentation #73

merged 1 commit into from
Jan 14, 2025

Conversation

jadudm
Copy link
Contributor

@jadudm jadudm commented Jan 5, 2025

PRs at this point do not yet have a full guide template, and are against a system that is not yet production-ready.

This commit is documentation only. It moves some pieces around, and most importantly, adds a document that describes how to add a new service. It is reasonably complete, having been guided by the bringing together of a new service (e2e), which could provide a starting point for a local/in-stack, repeatable path to end-to-end tests. That service is not complete, but standing up the shell/boilerplate guided this documentation.

Adding a story about how to add new services. This will help guide the
devs as we add new services to Jemison.
@jadudm jadudm changed the title Adding a new service Adding a new service documentation Jan 5, 2025
Copy link

@selfdanielj selfdanielj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doc moves make sense... started trying to follow the directions to setup a new service and left a few comments. Didn't quite finish but thought id post what i have so far.

docs/architecture/index.md Show resolved Hide resolved
@jadudm jadudm added this pull request to the merge queue Jan 14, 2025
Merged via the queue into main with commit caa261b Jan 14, 2025
3 checks passed
@jadudm jadudm deleted the jadudm/docs-0101 branch January 23, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: done
Development

Successfully merging this pull request may close these issues.

3 participants